EventRegisterSuccess에 대하여

파일/함수 이름의 길이가 너무 길지는 않은가?

• 우리가 이때까지 만들던 파일/함수의 이름보다는 길지만, 그럼으로 인해 뭔가 문제가 생길 것 같지는 않아 보입니다.
• 다만 EventRegister라는 이름이 새로운 이벤트를 등록(생성 또는 개설)한다는 뜻으로 혼동될 여지는 있어 보입니다. 그러나 적절한 대안을 찾지 못하겠네요… 나중에라도 더 좋은 이름을 생각해 보겠습니다.

우려사항

• EventRegister와 EventRegisterSuccess 페이지를 분리할 필요가 없어 보입니다.
• 페이지를 분리함으로 인해 최초 접속 페이지인 /register 라우트와 신청 성공 시 페이지인 /success 라우트가 생겼습니다. 이는 URL 입력으로 신청 성공 페이지에 진입할 수 있다는 의미도 되고, 이미 신청한 채로 최초 접속 페이지에 접속되었을 때 '취소하기' 버튼이 아닌 다른 요소들이 표시되리라는 의미도 됩니다.
• 아울러 상단 내비게이션 바에 있었던 일정 이름이 본문 영역으로 내려온 것도 UI 통일성의 관점에서 우려됩니다.

수정 제안사항

이에, 다음 두 가지를 제안합니다.

• EventRegister와 EventRegisterSuccess 통합 두 페이지를 통합하고, 사용자가 이미 이 일정을 신청했는지 여부와 사용자가 신청에 성공한 직후의 상태인지 여부를 판단하여 조금씩 다른 요소들을 표시하는 방식을 제안드립니다.
• 신청 완료 알러트 추가 신청이 완료되었다는 문구를 알러트 방식으로 표시하면 어떨까 합니다. 사용자가 일정 신청에 성공한 직후 일정 상세 페이지로 돌아왔을 때 본문 영역 최상단에 알러트를 띄워두고, 일정 시간이 지나면 알러트가 사라지거나 새로고침할 때 알러트가 사라지는 방법을 고려해볼 수 있겠습니다.

작업 방식

• 다만 제가 제안한 수정사항이 규모가 커 보여서, 우선 준엽 님의 의견을 여쭙습니다. 저도 단지 제안인 것이라… 편하게 거절하시거나 비판해주셔도 좋습니다!
• 만약 이 제안이 괜찮다고 생각하시면 제가 이 작업을 맡아서 처리하겠습니다.
• 중간총회까지 얼마 남지 않은 시점이라 코드-피드백-배포 루프를 빠르게 하는 게 좋을 것 같아, 우선 이 PR를 병합하겠습니다. 고생하셨습니다!